Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Week2] - seungwoo #37

Merged
merged 4 commits into from
Dec 20, 2024
Merged

[Week2] - seungwoo #37

merged 4 commits into from
Dec 20, 2024

Conversation

Gonzo-Dev-lab
Copy link
Collaborator

24-25 GDG 김승우
week2 스터디 과제 Pull request

junyeon00 and others added 3 commits November 22, 2024 17:06
Copy link
Member

@kimhamyong kimhamyong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

task3:

  • 현재 파라미터 그리드에서 가능한 조합의 수가 많아져서 계산 시간이 너무 오래걸리고 있습니다!
    GridSearchCV 대신 RandomizedSearchCV 사용: 무작위로 하이퍼파라미터를 탐색하는 방법을 추천드립니다.
  • 부트스트래핑을 100번 반복하여 개별 DecisionTreeRegressor를 학습시키고 있는데, 이미 RandomForestRegressor는 내부적으로 부트스트래핑과 앙상블을 수행하므로 별도로 구현할 필요가 없습니다.
  • make_moons 데이터셋은 이진 분류 문제입니다. 그러나 회귀 모델을 사용하고 있어 정확도가 낮게 나오고 있습니다.

수고하셨습니다.

Copy link
Collaborator

@giyomdabin giyomdabin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

task1, task2: 코랩에서 실행해본 결과, 코드가 수행되지 않습니다.

task1, task2 모두 특성 이름과 타깃 이름을 출력하는 과정에서 오류가 발생합니다.
문법을 다시 한 번 확인하신 후, 다시 업로드 해주세요!

감사합니다.

task1, task2 문법 수정 및 task3 모델 수정
@Gonzo-Dev-lab
Copy link
Collaborator Author

문법 문제 해결과 피드백 주신 부분 일부 수정 완료했습니다.

@giyomdabin
Copy link
Collaborator

task1, task2: 확인한 결과, 모두 잘 수행하셨습니다. 수고하셨습니다!

@giyomdabin giyomdabin merged commit 90da8fd into week2 Dec 20, 2024
@giyomdabin giyomdabin deleted the week2-seungwoo branch December 20, 2024 14:35
@kimhamyong kimhamyong changed the title Week2 seungwoo [Week2] - seungwoo Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants